Skip to content

Workaround for CI failure due to Data.bytes not being present.#1521

Merged
grynspan merged 2 commits into
mainfrom
jgrynspan/workaround-for-data-rawspan-ci-failure
Feb 3, 2026
Merged

Workaround for CI failure due to Data.bytes not being present.#1521
grynspan merged 2 commits into
mainfrom
jgrynspan/workaround-for-data-rawspan-ci-failure

Conversation

@grynspan

@grynspan grynspan commented Feb 3, 2026

Copy link
Copy Markdown
Contributor

swift-ci still uses an older Xcode version with an older OS SDK that does not include Data.bytes. As a result, Swift Testing fails to build for Apple platforms. Work around the issue by getting an unsafe buffer from the Data and using the buffer's RawSpan instead of the data's.

Resolves rdar://169480914.

Checklist:

  • Code and documentation should follow the style of the Style Guide.
  • If public symbols are renamed or modified, DocC references should be updated.

@grynspan grynspan added this to the Swift 6.4.0 (main) milestone Feb 3, 2026
@grynspan grynspan self-assigned this Feb 3, 2026
@grynspan grynspan added bug 🪲 Something isn't working workaround Workaround for an issue in another component (may need to revert later) build 🧱 Affects the project's build configuration or process labels Feb 3, 2026
@grynspan grynspan marked this pull request as ready for review February 3, 2026 16:46
@grynspan

grynspan commented Feb 3, 2026

Copy link
Copy Markdown
Contributor Author

Comment thread Sources/Testing/Support/JSON.swift
@grynspan

grynspan commented Feb 3, 2026

Copy link
Copy Markdown
Contributor Author

Wasm issues are known/unrelated.

@stmontgomery stmontgomery left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense, and my only concern is whether there are other failures lurking after this one. The Swift CI run should confirm

@grynspan

grynspan commented Feb 3, 2026

Copy link
Copy Markdown
Contributor Author

That's the only use of Data.bytes specifically (I checked) outside our test target (which doesn't need to build in the affected CI context).

AFAIK other stuff should be fine, but we'll find out.

…/adopt-span"

This reverts commit 4e8362c, reversing
changes made to 33e3787.
@grynspan grynspan force-pushed the jgrynspan/workaround-for-data-rawspan-ci-failure branch from 54884c7 to fc183e8 Compare February 3, 2026 18:52
@grynspan

grynspan commented Feb 3, 2026

Copy link
Copy Markdown
Contributor Author

Toolchain build isn't 100% complete, but Swift Testing built successfully so that's all we were looking for.

swift-ci still uses an older Xcode version with an older OS SDK that does not
include `Data.bytes`. As a result, Swift Testing fails to build for Apple
platforms. Work around the issue by manually constructing a `RawSpan` in the
relevant place.

Resolves rdar://169480914.
@grynspan grynspan force-pushed the jgrynspan/workaround-for-data-rawspan-ci-failure branch from fc183e8 to 70769fb Compare February 3, 2026 18:56
@grynspan grynspan merged commit e20dc82 into main Feb 3, 2026
22 of 24 checks passed
@grynspan grynspan deleted the jgrynspan/workaround-for-data-rawspan-ci-failure branch February 3, 2026 19:22
grynspan added a commit that referenced this pull request Feb 11, 2026
…for-data-rawspan-ci-failure"

This reverts commit e20dc82, reversing
changes made to 4e8362c.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug 🪲 Something isn't working build 🧱 Affects the project's build configuration or process workaround Workaround for an issue in another component (may need to revert later)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants